Skip to content

default port for bacnet; ensure INT for port and mask; fixing default BACnet config#2096

Open
majuss wants to merge 1 commit into
thingsboard:masterfrom
majuss:netmask_int_only
Open

default port for bacnet; ensure INT for port and mask; fixing default BACnet config#2096
majuss wants to merge 1 commit into
thingsboard:masterfrom
majuss:netmask_int_only

Conversation

@majuss

@majuss majuss commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

default BACnet config:

  • ensure mask and port are integer
  • application IP cannot be 0.0.0.0 when mask is /24, set a plausible IP

bacnet_connector:

  • ensure Port and Mask are read as Integer, everything else will throw errors (string doesnt make any sense)
  • ensure default BACnet port even when parameter port is completely skipped

bacnet_tests:

  • adjusted tests to integer

@majuss

majuss commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

@samson0v can you please have a look. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Connector:BACnet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@majuss